pixbuf-engine: Clean up error conditions and destructors

Simplify the error checks and move all common behaviour into a utility
function.

https://bugzilla.gnome.org/show_bug.cgi?id=712536

pixbuf-engine: Improve ThemePixbuf clean up functions

Make theme_pixbuf_destroy() NULL-safe like g_free(), and add a clear
function in the spirit of the g_clear_* family of functions.

https://bugzilla.gnome.org/show_bug.cgi?id=712536

Themes with specified gap images but no actual file crash.

https://bugzilla.gnome.org/show_bug.cgi?id=712536

x11: Unset GdkX11DeviceXI2 scroll valuators on device change

Scroll valuators were being just appended again and again, leading
to 1) a growing memory issue anytime a device changed 2) the first
scroll valuators to stay permanent on the application lifetime, as
the first stored valuators would always match.

https://bugzilla.gnome.org/show_bug.cgi?id=705203

build: bump requirement on gobject-introspection to 1.39.0

https://bugzilla.gnome.org/show_bug.cgi?id=713044

wayland: Sync transient-for on xdg-surface show as well...

wayland: Allow set_title after initial showing

and fix the ordering of title / app_id

Update xdg-shell.xml

printing: Don't wake up Avahi printers unnecessarily

Use info available in Avahi TXT records for creation of gtk printer
and request details when needed (through gtk_printer_request_details()).
If there is a printer advertised on Avahi by a remote CUPS server
try to get its PPD file at first or get its capabilities through an IPP
request if it fails.

https://bugzilla.gnome.org/show_bug.cgi?id=712751

combobox: Set GDK_SCROLL_MASK on the eventbox in appears-as-list mode

This is so the scroll wheel works in both modes, and there's no behavioral
change depending on a style property.

https://bugzilla.gnome.org/show_bug.cgi?id=702663

Update xdg-shell.xml

wayland: Don't assert fail in DND

This needs completion, sure thing, but let's try to just not fall
flat on our face first.

wayland: Don't recreate the gtk_surface on every show

It's illegal.

Revert "stack: respect gtk-enable-animations setting"

It already respected enable-animations.

This reverts commit fb9ff442997f02ee2d6ad0eae0321a961e96f56f.

wayland: Set DBus properties after we've constructed the xdg_surface

wayland: Fix invalid cast in transient_for

treeview: Use CSS property "border-top-color" to draw grid lines

This allows themes to customize the grid line colors in treeviews.

https://bugzilla.gnome.org/show_bug.cgi?id=415415

stack: respect gtk-enable-animations setting

https://bugzilla.gnome.org/show_bug.cgi?id=712632

revealer: Respect gtk-enable-animations setting

https://bugzilla.gnome.org/show_bug.cgi?id=712632

Bump version

3.11.2

reftests: Fix rendering issues with border-image reftest

When downscaling images, Cairo apparently uses algorithms different
enough to make this test trip over. So add the downscaled image as the
reference instead of downscaling the previous reference image.

Fixes the border-image reftest. For real now.

reftests: Fix border-image-repeat test

The new downscaling code in Cairo doesn't allow this test, so we remove
the CSS that made the border downscaled.

So the test does test less now, but it still tests the repeat modes of
border images.

gdk: Ignore crossings generated by passive grabs when resetting scroll axes

Passive grabs may take pointer focus out of the application, even though
the pointer didn't leave the window, but those events still trigger resetting
of the scroll axes. This is most visible with compiz, and possibly other
reparenting WMs, where passive grabs happen on the WM-managed window that
is a parent of the application toplevel.

As it is not possible to have scrolling happening on the timespan a passive
grab takes action, it is entirely safe for GTK+ to assume none happened if
it gets a crossing event of that nature.

https://bugzilla.gnome.org/show_bug.cgi?id=699574#c33

wayland: Replace wl_shell_surface with xdg_shell

Fix missing return and closing brace.

https://bugzilla.gnome.org/show_bug.cgi?id=712514

GtkBuilder: Make IDs optional

One requirement of .ui files is that each object must have an ID,
even if it is never referred to or directly loaded from the code.
This makes editing .ui files much more onerous than it has to be,
due to the frequent need to invent new IDs, while avoiding
clashes.

This commit makes IDs optional in the XML. They only need to
be provided for objects which are referred to or explictly loaded
from the code. Since GtkBuilder needs IDs for its own internal
accounting, we create IDs of the form ___object_N___ if not
specified in the XML.

https://bugzilla.gnome.org/show_bug.cgi?id=712553

Document GTK_THEME

Fix doc build

gtkwindow: Fix cast warning

gtkwidget: Fix up some deprecation warnings

Updated POTFILES.skip

x11: Add extra debug spew on XI2 crossing and button events

This information will be useful in case someone stumbles on a situation
similar to https://bugzilla.gnome.org/show_bug.cgi?id=699574, so we can
figure out where do the crossing events come from or go to easily.

gdk: be more selective resetting scroll events

It's been reported in several applications that scrolling feels jerky
since commit cc7b3985b3e31.

Investigation reported that the combination of passive 4-7 button grabs
on the toplevel and the presence of native subwindows might trigger
too often crossing events from the child window to the toplevel and
back as scroll "buttons" trigger the passive grab. Those crossing events
would reset the scroll valuators rendering scrolling from jerky on
touchpads (where there's intermediate smooth events between the emulated
button ones) to ineffective on regular mouse wheels (where the crossing
event would reset the valuators right before the single smooth scroll
event we get is delivered)

So, only reset scroll valuators when the pointer enters the toplevel
(we only care about this when the pointer is on the window after it's
been possibly scrolling somewhere else), and it doesn't come from an
inferior.

The situations where this happened varied though, the native subwindow
could be one created explicitly by the application, or created indirectly
through gdk_window_ensure_native(). The latter was mainly the case for
evolution (through gtk_selection_set_owner()) and any GtkScrolledWindow
under the oxygen-gtk3 theme (through gdk_window_set_composited())

https://bugzilla.gnome.org/show_bug.cgi?id=699574

Use window icons for fallback

When the window has an icon, use it for the fallback menu
button.

Add the new exampleapp to the docs

Add another example application

This new step sets the headerbar as titlebar, and enables
fallback for the app menu.

Small doc improvements

Fix fallback menubar size allocation

When the menubar inserted by GtkApplicationWindow is the widest
widget in a csd window, its allocation gets cut short. Fix this
by taking the decoration size into account while calculating
the size request (it is implicitly taken into account in the
size allocation phase by _gtk_window_set_allocation).

GtkWindow: better app menu fallback for CSD

Do the menubutton for app menu fallback ourselves in GtkWindow
for the csd, non-custom titlebar case. This fits better with
the way we handle other title buttons. Themes have control
over the placement of this button by placing menu in the
decoration-button-layout style property.

Trivial cleanup

Trivial whitespace fix

Updated Polish translation

Updated Spanish translation

Updated Spanish translation

Test headerbar app menu fallback

Make the sunny example useful by giving it a header bar
with app menu fallback. To test this under gnome-shell,
set APP_MENU_FALLBACK=1 in the environment.

GtkApplicationWindow: Avoid double fallback for the app menu

If we have a header bar which has the app menu fallback
enabled, don't add the app menu to the fallback menubar.

GtkHeaderBar: Add a fallback app menu implementation

Allow showing the fallback app menu with a menu button
in the header bar. Applications have to explicitly enable
this by calling gtk_header_bar_set_show_fallback_app_menu.

GtkWindow: Make titlebar widget available privately

This will be used in GtkApplicationWindow in the
following commits.

GtkButton: don't propagate double-clicks

It is basically always unexpected to have a container handle
double-clicks on a widget that handles single-clicks itself.

Updated Greek translation

Updated Brazilian Portuguese translation

tests: Make testinput example not compress events

testgtk: Don't use GDK_POINTER_MOTION_HINT_MASK anymore

gdk: Document event compression better

Advise against using GDK_POINTER_MOTION_HINT_MASK.
Add pointers to gdk_window_set_event_compression() to the documentation.

GtkAboutDialog: Fix a corner case of link highlighting

GtkAboutDialog highlights emails written as <...> and
urls written as http://... . gnome-terminal manages to
put <http://...> into its license text, which sadly
confuses the parser into running evolution on http://...

Fix things up far enough that <http://...> is now
recognized as url, and only the part inside the <> is
underlined (for email addresses, we include the <> in
the underline).

Win32: Set the shows-desktop platform setting

Win32 shows icons on the desktop, so set shows-desktop
to TRUE here.

Quartz: Set the shows-desktop platform setting

OS X shows icons on the desktop, so set this setting
to TRUE here.

GtkSettings: make show-desktop default to FALSE

This broke tests.

stylecontext: Revert part of previous commit

Commit 719dd636a9da679ea08a9644a8cccfe7d6f145d7 replaces
margin-left/right with margin-start/end. CSS does not have
margin-start/margin-end properties, the sed script was a bit overeager.

Fwiw, CSS implements RTL margin styling via :dir(rtl) selectors.

GtkPlacesSidebar: don't unref an object we don't own

gtk_settings_get_default() doesn't return a reference, so don't unref it
during dispose.

Replace all margin-left and margin-right with margin-start and margin-end

https://bugzilla.gnome.org/show_bug.cgi?id=710238

GtkWidget: Add margin-start and margin-end properties

Add margin-{start,end} and gtk_widget_{get,set}_margin_{start,end}
and drop margin-{left,right} and gtk_widget_{get,set}_margin_{left,right}.

margin-{start,end} handle right also in RTL.

https://bugzilla.gnome.org/show_bug.cgi?id=710238

GtkPlacesSidebar: use shell-shows-desktop setting

Use the just-added shell-shows-desktop GtkSetting to determine if we
should also show the "Desktop" folder in the places sidebar.

https://bugzilla.gnome.org/show_bug.cgi?id=712302

Add a GtkSetting for 'shell-shows-desktop'

Add a GtkSetting for whether the desktop shell is showing the desktop
folder icons.

This is on by default because most desktop shells do show the icons on
the desktop.  We already have a patch in gnome-settings-daemon to bind
this to the org.gnome.desktop.background show-desktop-icons GSettings
key which is off by default on GNOME.

https://bugzilla.gnome.org/show_bug.cgi?id=712302

Updated Polish translation

broadway: Improve touch events

We now respect core pointer grabs, and we do the pointer emulation
detection on the browser side.

Add print to PDF to recent manager

Printing a file with to PDF/etc. with the virtual printer option doesn't add
the file to the list of recently used files. It should be there, so I can
easily access it.

https://bugzilla.gnome.org/show_bug.cgi?id=668598

GtkPlacesSidebar: Show unmounted bookmarks

The GtkFileInfo is only used if there's no bookmark name,
or to get the icon for the remote share or local folder.

https://bugzilla.gnome.org/show_bug.cgi?id=711548

gtktextattributes: Only use transparent union for g-ir-scanner

https://bugzilla.gnome.org/show_bug.cgi?id=711158

Clean incorrect "_" use in the french translation

https://bugzilla.gnome.org/show_bug.cgi?id=712213

broadway: Support ipad on-screen keyboard

We add a custom im module for broadway that calls some broadway
specific APIs to show/hide the keyboard on focus in/out. We then forward this
to the browser, and on the ipad we focus an input field to activate
the keyboard.

broadway: Send focus out events

broadway: Fix focus handling with touch events

gdk/win32/gdkevents-win32.c: Fix typo

...to fix the build on Windows.

GtkApplicationWindow: avoid mis-allocation

In the fallback case, we were adding the border width of the
window twice, causing the content to be mis-positioned and
cut off in some cases.

https://bugzilla.gnome.org/show_bug.cgi?id=710909

GtkMenuTracker: set action namespace correctly

gtk_menu_tracker_add_items() fetched the action-namespace from the menu
item, but didn't pass it into gtk_menu_tracker_section_new() when its
internal namespace was still NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=712164

broadway: Support window resize/drag via touch

GtkStack: add transition-running property

This is useful to know when the transition is done we can
remove the child from the stack.

https://bugzilla.gnome.org/show_bug.cgi?id=711494

broadway: Add initial touch event support

This seems to get something going on an ipad, but some events seem
to get swallowed. For instance, window dragging doesn't work.

broadway: Disable all kinds of viewport scrolling and zooming

broadway: Raise window on first click.

broadway: Add raise/lower ops

broadway: use 'click-to-focus' approach instead of 'focus-follows-mouse'

The broadway backend would move the focus from one window to another based on
where the mouse was (i.e. 'focus-follows-mouse' approach). Handling the focus
this wait didn't play well with widgets which rely on focus-in-event and
focus-out-event, like the GtkEntry when using a completion popup window, see
e.g:
  https://bugzilla.gnome.org/show_bug.cgi?id=708984

So instead, setup broadway to require a click in a window to move the focus
(i.e. 'click-to-focus' approach):

 * The implicit GDK_FOCUS_CHANGE events that were generated upon reception of
   BROADWAY_EVENT_ENTER or BROADWAY_EVENT_LEAVE are removed.

 * The broadway daemon will now keep track of which is the focused window

 * Whenever the daemon detects an incoming BROADWAY_EVENT_BUTTON_PRESS, it will
   trigger the focused window switch, which sends a new BROADWAY_EVENT_FOCUS to
   the client, specifying which windows holds the focus.

 * Upon reception of a BROADWAY_EVENT_FOCUS, the client will generate a new
   GDK_FOCUS_CHANGE.

 * gdk_broadway_window_focus() was also implemented, which now requests the
   focus to the broadway server using a new BROADWAY_REQUEST_FOCUS_WINDOW.

This is based on an initial patch from Aleksander Morgado <aleksander@lanedo.com>.

Handle recursion from motion event handlers

If a motion event handler (or other handler running from the flush-events
phase of the frame clock) recursed the main loop then flushing wouldn't
complete until after the recursed main loop returned, and various aspects
of the state would get out of sync.

To fix this, change flushing of the event queue to simply mark events as
ready to flush, and let normal event delivery handle the rest.

https://bugzilla.gnome.org/show_bug.cgi?id=705176

treeview: fix dragging of columns in mutter

Don't recurse the mainloop in _gtk_tree_view_column_start_drag().
It doesn't serve any discernible purpose, and recursing the
mainloop from the flush-events phas of the frame clock breaks
frame synchronization with mutter.

https://bugzilla.gnome.org/show_bug.cgi?id=705176

gtkpixelcache: Fix possibly ignoring queued redraw

If a queue_redraw() (invalidating a region, or the whole widget) was
called from the draw() call, it could get ignored if surface_dirty
existed, as it would then be updated, but destroyed right at the end of
the _gtk_pixel_cache_repaint(), leading the next call to
_gtk_pixel_cache_draw() have its call to repaint() be a no-op
(since there's no surface_dirty) and then simply draw from (non
updated) surface.

Signed-off-by: Olivier Brunel <jjk@jjacky.com>
https://bugzilla.gnome.org/show_bug.cgi?id=711545

Fix warning with event compression on a destroyed window

Fix a critical message when we try to compress events for a window
that was already destroyed.

Updated Greek translation

Progress bar: Handle 'no more pulse'

Before smooth animation, stopping the pulse would abruptly stop
the bouncy bar. This change makes it so that we slow down for
a bit, and then stop.

widget-factory: Make the activity mode more complete

Test the 'no more pulse' situation. When the slider goes all
the way to the right towards slow, we remove the pulse timeout.

Update drawing model docs

Include material from Alex' blog post about the modern GTK+
rendering model.

Make activity mode progress bars animate better

Use a tick callback and move the block each frame, instead
of making it jump only when gtk_progress_bar_pulse() is called.

Make the progress bar speed variable

Arrange for the scales to control the speed of the activity
mode progress bar in gtk3-widget-factory. This will be useful
in demonstrating smoother progress bar animation in the future.

Move a function around

_Gtk_file_chooser_label_for_file is not actually
used in gtkfilechooserdefault.c anymore, so move
it to a better place.

Silence a compiler warning

Filter out duplicate application shortcuts

The file chooser used to do this, the places sidebar
still needs to learn it.

https://bugzilla.gnome.org/show_bug.cgi?id=711636

Test duplicate shortcuts

Add an application shortcut for the Music folder in testfilechooser.
The file chooser used to filter duplicates out, but this broke
in the sidebar merge.

GtkFileChooser: propagate local-only to sidebar

This fixes a regression from the sidebar merge.

GtkPlacesSidebar: Add local-only

This is necessary to implement the filechooser property
of the same name.

https://bugzilla.gnome.org/show_bug.cgi?id=711574